Skip to content

[WIP] power-assert v2 #27

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 115 commits into
base: master
Choose a base branch
from
Open

[WIP] power-assert v2 #27

wants to merge 115 commits into from

Conversation

twada
Copy link
Member

@twada twada commented Nov 23, 2018

Experimental implementation of power-assert v2 transpiler side.

twada added 30 commits January 15, 2019 09:48
@twada
feat: first PoC implementation of power-assert-2 transpiler
b95c5da
@twada
test: add tests and jsdoc for ArgumentRecorder
541a901
@twada
chore: set default Promise wrapper status as "pending"
8f6f281
@twada
chore(empower-core): PoC implementation of power-assert-2 function re…
33b6f7e 
…sult capturing
@twada
chore: pass callee of assertion to recorder to enable runtime feature…
7da014f 
… detection
@twada
chore: PoC of assertion recorder with assignment
2324da5
@twada
docs: adding JSDoc comments and type declarations
415c95a
@twada
refactor: rename variable
b477fd8
@twada
refactor: separate assignment from node creation
0d665aa
@twada
feat: put AssertionRecorder in place of assertion message argument
d3404ee
@twada
feat: return ArgumentRecorder itself since we have callee for runtime…
7e0bf25 
… feature detection
@twada
feat: pass AssertionMetadata to ArgumentRecorder so now runtime side …
693e05f 
…can retrieve metadata from args
@twada
refactor: AssertionRecorder also acts like ArgumentRecorder to simpli…
ea87680 
…fy client code
@twada
refactor: rename AssertionRecorder to AssertionMessage
4a4af2e
@twada
chore: shorten generated name a bit
b77118f
@twada
refactor: generalize optional message handling
84a342a
@twada
fix: use currentNode.arguments.length to calculate number of actual a…
aecab0d 
…rguments
@twada
feat: enhance message even if it is a string literal
4675ab8
@twada
feat: support assert.rejects and assert.doesNotReject
4a10799
@twada
feat: support assert.throws, assert.doesNotThrow
8b23eb8
@twada
chore: update call-matcher to ^1.1.0
c0277f7
@twada
refactor: reduce number of matching attempts
61ae79c
@twada
feat: proxy functions if and only if it is a block argument
1d84086
@twada
refactor: separate argument capturing from intermediate Node capturing
6c69ec0 
to reduce generated code
@twada
refactor: embed argument metadata to move block handling
8b2c3c2
@twada
feat: embed config object into code to give more hint to runtime side
2b0a87b
@twada
chore: fix delegation
7ccd6ed
@twada
fix: dealing with undefined callee (there's no such method)
714e561 
TypeError: Cannot read property '_empowered' of undefined
@twada
test: update fixtures
b0bd7f9
@twada
chore: sync with latest visitorKeys
ba6cce7
twada added 30 commits February 10, 2019 08:17
@twada
refactor: use positional parameters and shorthand notation for shorte…
3b3adee 
…r generated code
@twada
refactor: invoke LocationDetector only once
534ebec
@twada
refactor: prefer side effect free function
b135e49
@twada
chore: count propsNode.properties
065420d
@twada
refactor: generate filepath for each assertion since input code would…
76314a0 
… be concatenated
@twada
Merge branch 'metadata-function' into power-assert-2
cd5a77d
@twada
test: update babel and sync with latest visitorKeys
caa29cf
@twada
refactor: use SpreadElement to gain readability
78c37bb
@twada
refactor: inline shorthand object expressions to gain readability
a2140b2
@twada
fix: use Object.assign since Node6 does not support Spread Properties
0153a35
@twada
fix: use Object.assign since Node6 does not support Spread Properties
d6a4768
@twada
chore(package): upgrade espower-location-detector
618076b
@twada
docs: update JSDoc
7c614ef
@twada
chore: update package-lock.json
1d25fac
@twada
refactor: prefer method call
fd13cb8
@twada
refactor: use parameter for declarations, argument for actual values
bb653c7
@twada
chore: add TODO
eb61e82
@twada
chore: per argument renderer configuration
9155a3b
@twada
chore: update package-lock.json
d36a98c
@twada
test: sync with latest visitorKeys
59cb2fb
@twada
fix: dealing with node not to be captured that appears immediately be…
a86ed54 
…neath assert
@twada
test: update expected output
257bdfc
@twada
docs: update jsdoc
92d9ef5
@twada
Merge branch 'case-skip-capturing-toplevel-expression' into power-ass…
bd3153a 
…ert-2
@twada
feat: embed instrumentation patterns as JSON
c13a70b
@twada
test: update expected output
ea55fba
@twada
feat: embed transpiler name and its version into output
1e8dd3e
@twada
test: update expected output
7e635e3
@twada
Merge branch 'embed-transpiler-version' into power-assert-2
77c4edd
@twada
chore(travis): add Node12, drop Node6 and Node11
658a969
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@twada